fix(pm-skills): use plain tag name for ref, not refs/tags/#1771
fix(pm-skills): use plain tag name for ref, not refs/tags/#1771Avyayalaya wants to merge 2 commits into
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
main, but PRs should target staged.
The main branch is auto-published from staged and should not receive direct PRs.
Please close this PR and re-open it against the staged branch.
You can change the base branch using the Edit button at the top of this PR,
or run: gh pr edit 1771 --base staged
github-actions
Bot
added
external-plugin
|
🟡 Contributor Reputation Check: MEDIUM risk
Maintainers: please review this contributor before merging. |
github-actions
Bot
added
the
needs-review:MEDIUM
`apm install pm-skills@awesome-copilot` invokes `git clone --depth=1 --branch=<ref>` against the source repo. `git clone --branch` rejects `refs/tags/v2.1.0` ("Remote branch not found in upstream origin") and only accepts the plain tag name.
Matches the convention used by the other entries in this file — `chrome-devtools-plugin` uses `chrome-devtools-mcp-v1.0.1`, `ai-ready` uses a SHA. `pm-skills` was the only entry with the `refs/tags/` prefix, inherited from the placeholder example in the external-plugin issue form.
Tested locally: with the fix applied, `apm install pm-skills@awesome-copilot --target claude` resolves cleanly (12 skills + MCP server + 2 hooks integrated).
Original submission: github#1767 / github#1770
Avyayalaya
force-pushed
the
fix/pm-skills-ref-format
branch
from
90bcf30 to
2763d40
Compare
|
Force-pushed a clean version of the branch — the previous diff included Unicode encoding artifacts on three other entries (ai-ready, microsoft-events, whatidid) caused by a Python JSON re-encode pass on my end. Fixed with a surgical sed edit; the diff is now a single one-line change to the pm-skills ref. Apologies for the noise. |
|
🟡 Contributor Reputation Check: MEDIUM risk
Maintainers: please review this contributor before merging. |
github-actions
Bot
added
branched-main
|
🟡 Contributor Reputation Check: MEDIUM risk
Maintainers: please review this contributor before merging. |
1 participant
Single-line fix in
plugins/external.jsonfor thepm-skillsentry. Thesource.refwasrefs/tags/v2.1.0(inherited from the external-plugin issue form's placeholder example). APM invokesgit clone --branch=<ref>against the source repo, which rejects therefs/tags/prefix:Other entries in this file use plain tag names (
chrome-devtools-plugin→chrome-devtools-mcp-v1.0.1) or SHAs (ai-ready,sonarqube);pm-skillswas the only entry with therefs/tags/prefix.Test
After this fix:
apm marketplace add github/awesome-copilot apm install pm-skills@awesome-copilot --target claude # → 12 skills + MCP server + 2 hooks integrated in ~35sOriginal submission: #1767 / merged-PR #1770.
Note for the issue-form template: the placeholder example reads
refs/tags/v1.2.3 or 0123456789abcdef.... Therefs/tags/form is technically a valid git ref but doesn't survive thegit clone --branchround-trip APM uses. Worth either changing the placeholder or normalizing the ref in intake validation — happy to send a follow-up PR if helpful.🤖 Generated with Claude Code